Skip to content

docs(playground): minor clean up in the playground's tabs#48

Merged
danielroe merged 3 commits into
nuxt-modules:mainfrom
azrikahar:minor-fixes
Aug 31, 2022
Merged

docs(playground): minor clean up in the playground's tabs#48
danielroe merged 3 commits into
nuxt-modules:mainfrom
azrikahar:minor-fixes

Conversation

@azrikahar

Copy link
Copy Markdown
Contributor

Minor clean up & consistency changes to the playground's tabs.

Tab1

  • remove add ionicon import as it seems to not be used in the tab
  • remove the class from <ion-toolbar> just as a clean up and I believe it's not currently in use

Tab2

  • remove ionicon imports and use them directly as they're auto imported
  • import UserPhoto type for showActionSheet
  • <ion-col>'s :key was using the photo object directly, updated it to use photo.filepath instead

Tab4

@vercel

vercel Bot commented Aug 31, 2022

Copy link
Copy Markdown

Someone is attempting to deploy a commit to a Personal Account owned by @danielroe on Vercel.

@danielroe first needs to authorize it.

@vercel

vercel Bot commented Aug 31, 2022

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
nuxt-ionic ✅ Ready (Inspect) Visit Preview Aug 31, 2022 at 2:45PM (UTC)

@danielroe danielroe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - thank you very much!

@danielroe danielroe changed the title refactor(playground): Minor clean up in the playground's tabs docs(playground): minor clean up in the playground's tabs Aug 31, 2022
@danielroe danielroe merged commit 3657e86 into nuxt-modules:main Aug 31, 2022
@azrikahar azrikahar deleted the minor-fixes branch August 31, 2022 14:59
@github-actions github-actions Bot mentioned this pull request Jun 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants